Skip to content

Conversation

@hannahTao
Copy link

…with tests and test data

Copy link
Contributor

@nrfulton nrfulton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please review these comments. Otherwise, LGTM.

def is_appellate_court(court_name: str) -> ValidationResult:
"""Determine if a court is an appellate court based on its name."""
# rule exceptions: the 2 appellate courts whose names do not include the below keywords
rule_exceptions = ["pennsylvania superior court", "pennsylvania commonwealth court"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Are you sure that these are the only two exceptions?
  2. Do we need to handle abbreviations of court names, or can we always expect that the full court name will be provided as input to this function?

def __init__(self, folder_path):
self.folder_path = folder_path
super().__init__(
description="The result should be an appellate court case.",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A citation of?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants